Added support for etf extraction by symbol #427
Closed
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Added support for etf extraction by symbol
As requested on this issue #425, a new feature has been added to the etf information extraction functionalty. Now it's possible to obtain the desired information by either indicating the etf name or symbol.
In order to differentiate wether it is the symbol or the name, a new
is_symbol
parameter has been added to the functions of etf extraction suchget_etf_recent_data()
,get_etf_historical_data()
andget_etf_information()
.As result, the way you can extract the information with name remains unchanged:
But if you want to retrieve the information by symbol the resulting method should look like this:
Note that the
is_symbol
parameter is optional and its default value is set toFalse
in order to maintain how the previous were used.All tests for this new functionality has been added to the tests scripts and has been passed without errors.